Skip to content

Conversation

@mnonnenmacher
Copy link
Contributor

No description provided.

@mnonnenmacher
Copy link
Contributor Author

Teams need write access to the repository before they can be used as code owners.

@sschuberth
Copy link
Contributor

Note that a downside of using teams might be that GitHub then only requires one team member to start a review, and from then on only that member is listed as a reviewer, and the team is removed. At least that's the behavior I see in ORT. The problem with this is that if the original reviewer does not complete the review / approve, then no other reviewers will be added back automatically.

@sschuberth
Copy link
Contributor

We actually had a discussion about this exact topic at OSS Summit Europe with @schr3gl3j and others, coming to the conclusion that keeping the explicit names is more transparent to contributors, and also allows for a more fine-granular configuration of code owners. So I'd actually propose to drop this PR, and also revert to explicit user names .in CODEOWNERS for ORT (core).

@mnonnenmacher
Copy link
Contributor Author

We actually had a discussion about this exact topic at OSS Summit Europe with @schr3gl3j and others, coming to the conclusion that keeping the explicit names is more transparent to contributors, and also allows for a more fine-granular configuration of code owners. So I'd actually propose to drop this PR, and also revert to explicit user names .in CODEOWNERS for ORT (core).

Fine with me.

@mnonnenmacher mnonnenmacher deleted the teams-codeowners branch August 31, 2025 17:25
@mnonnenmacher
Copy link
Contributor Author

@sschuberth I have made the change for ORT here: oss-review-toolkit/ort#10818

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants